Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: stabilize EIP-7702 #3427

Merged
merged 14 commits into from
Mar 26, 2025
Merged

feat: stabilize EIP-7702 #3427

merged 14 commits into from
Mar 26, 2025

Conversation

jxom
Copy link
Member

@jxom jxom commented Mar 9, 2025

No description provided.

Copy link

changeset-bot bot commented Mar 9, 2025

🦋 Changeset detected

Latest commit: 774d307

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
viem Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented Mar 9, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
viem 🛑 Canceled (Inspect) Mar 27, 2025 0:20am

Copy link
Contributor

github-actions bot commented Mar 9, 2025

size-limit report 📦

Path Size Loading time (3g) Running time (snapdragon) Total time
import * from 'viem' (esm) 67.21 KB (0%) 1.4 s (0%) 13.6 s (+49.93% 🔺) 15 s
const viem = require('viem') (cjs) 81.02 KB (0%) 1.7 s (0%) 16.4 s (-4.04% 🔽) 18 s
import { createClient, http } from 'viem' 4.25 KB (0%) 85 ms (0%) 717 ms (-7.49% 🔽) 802 ms
import * from 'viem/account-abstraction' 47.66 KB (0%) 954 ms (0%) 6.1 s (-52.36% 🔽) 7.1 s
import * from 'viem/accounts' 82.17 KB (0%) 1.7 s (0%) 6.1 s (-49.9% 🔽) 7.7 s
import { privateKeyToAccount } from 'viem/accounts' 20.69 KB (0%) 414 ms (0%) 2.6 s (-22.22% 🔽) 3 s
import { mnemonicToAccount } from 'viem/accounts' 27.01 KB (0%) 541 ms (0%) 5.8 s (+10.08% 🔺) 6.3 s
import * from 'viem/actions' 52.48 KB (0%) 1.1 s (0%) 8.9 s (+19.32% 🔺) 10 s
import { getBlockNumber } from 'viem/actions' 318 B (0%) 10 ms (0%) 248 ms (+181.72% 🔺) 258 ms
import * from 'viem/chains' 46.35 KB (0%) 928 ms (0%) 10.8 s (+56.39% 🔺) 11.7 s
import { mainnet } from 'viem/chains' 325 B (0%) 10 ms (0%) 320 ms (+1031.51% 🔺) 330 ms
import * from 'viem/chains/utils' 1.08 KB (0%) 22 ms (0%) 130 ms (-73.34% 🔽) 152 ms
import * from 'viem/ens' 46.17 KB (0%) 924 ms (0%) 2.9 s (-45.45% 🔽) 3.8 s
import { getEnsAvatar } from 'viem/ens' 22.44 KB (0%) 449 ms (0%) 6.2 s (+141.96% 🔺) 6.6 s
import * from 'viem/siwe' 31.57 KB (0%) 632 ms (0%) 3.5 s (-1.38% 🔽) 4.1 s
import { verifySiweMessage } from 'viem/siwe' 30.51 KB (0%) 611 ms (0%) 6.8 s (+99.89% 🔺) 7.4 s

Copy link

codecov bot commented Mar 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.66%. Comparing base (9929b2c) to head (774d307).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff            @@
##             main    #3427    +/-   ##
========================================
  Coverage   99.66%   99.66%            
========================================
  Files        1002      998     -4     
  Lines       76849    76623   -226     
  Branches     3308     3305     -3     
========================================
- Hits        76588    76363   -225     
+ Misses        239      238     -1     
  Partials       22       22            

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@jxom jxom requested a review from Copilot March 26, 2025 23:16
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR stabilizes EIP-7702 by updating documentation examples, changing experimental exports to their stable counterparts, and officially replacing the sponsor parameter with executor.

  • Updated code examples and documentation to reference stable exports from viem and viem/utils.
  • Adjusted parameter naming from sponsor to executor and updated related utilities.
  • Updated changesets to document removal of deprecated experimental exports.

Reviewed Changes

Copilot reviewed 90 out of 93 changed files in this pull request and generated no comments.

Show a summary per file
File Description
.changeset/odd-parents-crash.md Updated breaking change description for stable EIP-7702 exports.
site/pages/docs/eip7702/sending-transactions.md Revised examples showing construction of EIP-7702 transactions with updated API usage.
site/pages/docs/eip7702/contract-writes.md Updated examples for contract writes using updated EIP-7702 authorization logic.
.changeset/mighty-maps-dress.md Documented removal of deprecated ERC-6492 exports.
.changeset/new-seas-wash.md Documented removal of deprecated walletActionsErc7715 export.
.changeset/clever-mice-laugh.md Summarized stabilization of EIP-7702 with new actions and utility updates.
site/pages/docs/eip7702/signAuthorization.md Updated documentation for signAuthorization, switching from experimental to stable usage.
.changeset/dirty-trees-pay.md Documented removal of deprecated walletActionsEip5792 export.
site/pages/docs/eip7702/recoverAuthorizationAddress.md Updated import paths from experimental to stable utilities for recovering authorization addresses.
site/pages/docs/eip7702/prepareAuthorization.md Updated prepareAuthorization docs to reflect parameter name changes and new API paths.
Various site/docs pages (signTransaction, sendTransaction, prepareTransactionRequest, simulateContract, writeContract) Revised references to EIP-7702 docs and updated links from experimental to stable endpoints.
site/pages/docs/accounts/local/signTransaction.md Updated account method naming to reflect stabilized EIP-7702 API.
site/pages/docs/eip7702/hashAuthorization.md Updated imports for hashAuthorization from experimental to stable utilities.
Files not reviewed (3)
  • package.json: Language not supported
  • pnpm-lock.yaml: Language not supported
  • site/pages/docs/eip7702.mdx: Language not supported
Comments suppressed due to low confidence (1)

site/pages/docs/eip7702/signAuthorization.md:358

  • Duplicate 'to' property detected in the transaction object literal. Remove one of the 'to' keys to avoid unintended behavior.
  to: walletClient.account.address, // [!code ++]

@jxom jxom merged commit 2a5ded4 into main Mar 26, 2025
30 of 31 checks passed
@jxom jxom deleted the stable-7702 branch March 26, 2025 23:25
@github-actions github-actions bot mentioned this pull request Mar 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant